Skip to content

docs(core): add guidelines for naming CSS shadow parts#30931

Merged
brandyscarney merged 7 commits intofeature-8.8from
FW-6794-docs
Feb 27, 2026
Merged

docs(core): add guidelines for naming CSS shadow parts#30931
brandyscarney merged 7 commits intofeature-8.8from
FW-6794-docs

Conversation

@brandyscarney
Copy link
Member

Adds guidelines document establishing standardized naming conventions for CSS Shadow Parts in Ionic Framework components.

@vercel
Copy link

vercel bot commented Jan 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ionic-framework Ready Ready Preview, Comment Feb 27, 2026 3:39pm

Request Review

@brandyscarney brandyscarney marked this pull request as ready for review January 20, 2026 19:36
@brandyscarney brandyscarney requested a review from a team as a code owner January 20, 2026 19:36
@brandyscarney brandyscarney requested a review from gnbm January 20, 2026 19:36
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ShaneK ShaneK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have two minor change requests, but looks great! Awesome work!

brandyscarney and others added 3 commits February 27, 2026 10:29
Co-authored-by: Shane <shane@shanessite.net>
Co-authored-by: Shane <shane@shanessite.net>
@brandyscarney brandyscarney merged commit 1d7b286 into feature-8.8 Feb 27, 2026
49 checks passed
@brandyscarney brandyscarney deleted the FW-6794-docs branch February 27, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants